Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update email when linking respondent #2426

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rebecarubio
Copy link
Contributor

Description

This PR adds a dialog to allow the user to add the respondent email to the person that is going to be link as a new respondent.
It also introduces a new extra reducer in the Survey store that updates de submission list when a person is updated.

Screenshots

Video.sin.titulo.Hecho.con.Clipchamp.1.mp4

Changes

  • Adds an if statement in SurveySubmissionsList when a person to be linked as a respondent doesn't have an email
  • Adds a SurveyDialogLink component to display a Dialog when the if statement is triggered
  • Adds an extra reducer in Survey store to update the submissions list when personUpdated action is triggered.

Notes to reviewer

None

Related issues

Resolves #2269

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add missing email address when linking survey response
1 participant